Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Backport JDK-8280476 fix #191

Open
wants to merge 1 commit into
base: jbr17.469
Choose a base branch
from

Conversation

rogerhu
Copy link

@rogerhu rogerhu commented Dec 5, 2022

This bug (https://bugs.openjdk.org/browse/JDK-8280476) is triggered when building against Xcode 13.3. I see the exact same error message and realized that this commit needed to be added.

During or after building the image, I see:

/Users/rogerh/Development/JetBrainsRuntime/build/macosx-aarch64-server-release/jdk/bin/java --version
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  Internal Error (assembler_aarch64.hpp:248), pid=63683, tid=9475
#  guarantee(val < (1ULL << nbits)) failed: Field too big for insn
#
# JRE version:  (17.0.5) (build )
# Java VM: OpenJDK 64-Bit Server VM (17.0.5-internal+0-adhoc.rogerh.JetBrainsRuntime, mixed mode, sharing, tiered, compressed oops, compressed class ptrs, g1 gc, bsd-aarch64)
# No core dump will be written. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again
#
# An error report file with more information is saved as:
# /Users/rogerh/Development/JetBrainsRuntime/build/macosx-aarch64-server-release/make-support/failure-logs/hs_err_pid63683.log
#

There are also sprintf error warnings that I was able to suppress to get things to compile with Xcode 13:

+++ b/make/hotspot/gensrc/GensrcAdlc.gmk
@@ -58,6 +58,8 @@ ifeq ($(call check-jvm-feature, compiler2), true)
 
   ADLC_CFLAGS += -I$(TOPDIR)/src/hotspot/share
 
+  ADLC_CFLAGS += -Wno-deprecated-declarations
+
   # Add file macro mappings
   ADLC_CFLAGS += $(FILE_MACRO_CFLAGS)

Along with passing in --with-extra-cxxflags="-Wno-deprecated-declarations" when running the configure script.

@rogerhu rogerhu changed the title Allow JDK Backport JDK-8280476 fix Dec 5, 2022
@rogerhu rogerhu changed the base branch from jbr17 to jbr17.469 December 5, 2022 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant